Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reword and reformat README for idiomaticity and clarity #341

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

walking-octopus
Copy link

@walking-octopus walking-octopus commented Feb 15, 2025

  • The excessive HTML use made it difficult to read as plaintext, which is counter to Markdown's philosophy. A lot of HTML tags were used when equivalent Markdown syntax existed. For the remaining HTML, I tried to make it semantically sensible, minimizing attempts to control the presentation. In the process, it did remove manual heading IDs, but doesn't GitHub generate those automatically? I might need to revert headings back to tags.

  • Some names were misspelled, some units used non-standard symbols, many phrases sounded off, the tone at times sounded too excessive. I tried to use my best judgment to guess the intended meaning. If I got anything wrong, let me know.

@Azure-Tang
Copy link
Contributor

Thx a lot for reformatting our readme! 🌹

@Azure-Tang
Copy link
Contributor

It seems our second video, which introducing tabby, is removed?

And there are two "Quick Start" sections now, which the second should be "Brief Injection Tutorial"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants